Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[1839] Fix ConflictingPublicNetworkAccessAndVirtualNetworkConfiguration #112

Conversation

saliceti
Copy link
Member

@saliceti saliceti commented Aug 13, 2024

Context

Related to issue hashicorp/terraform-provider-azurerm#26098

We must ignore changes to the public_network_access_enabled argument as it becomes enforced from azurerm version v3.105.0.
This will allow us to upgrade azurerm in all the repositories. Then we will be able to set public_network_access_enabled to false and remove this ignore_changes.

Changes proposed in this pull request

Add public_network_access_enabled to ignore_changes lifecycle

Guidance to review

See testing in https://hedgedoc.teacherservices.cloud/GfdrzaeGRKGO49k69ANvrQ?edit

Checklist

  • I have performed a self-review of my code, including formatting and typos
  • I have cleaned the commit history
  • I have added the Devops label
  • I have attached the pull request to the trello card

@saliceti
Copy link
Member Author

Related to issue hashicorp/terraform-provider-azurerm#26098

We must ignore changes to the argument as it becomes enforced from
azurerm version v3.105.0.
This will allow us to upgrade azurerm in all the repositories. Then we
will be able to set public_network_access_enabled to false and remove
this ignore_changes.
@saliceti saliceti force-pushed the 1839-bug-conflictingpublicnetworkaccessandvirtualnetworkconfiguration branch from bd32b98 to 779ab63 Compare August 13, 2024 13:30
@saliceti saliceti marked this pull request as ready for review August 13, 2024 13:35
@saliceti saliceti merged commit 73a2937 into main Aug 14, 2024
3 checks passed
@saliceti saliceti deleted the 1839-bug-conflictingpublicnetworkaccessandvirtualnetworkconfiguration branch August 14, 2024 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants